[PT FE] Refactor PixelShuffle operations#33743
Open
mvafin wants to merge 3 commits intoopenvinotoolkit:masterfrom
Open
[PT FE] Refactor PixelShuffle operations#33743mvafin wants to merge 3 commits intoopenvinotoolkit:masterfrom
mvafin wants to merge 3 commits intoopenvinotoolkit:masterfrom
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR refactors the PixelShuffle operations and simplifies the negation operation in the PyTorch frontend by leveraging existing OpenVINO operations instead of custom implementations.
Changes:
- Replaced custom
translate_negimplementation with a direct mapping toopset10::Negative - Refactored
translate_pixel_shuffleandtranslate_pixel_unshuffleto useDepthToSpaceandSpaceToDepthoperations - Simplified
translate_channel_shuffleto use theShuffleChannelsoperation
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| src/frontends/pytorch/src/op_table.cpp | Updated operation mappings to use translate_1to1_match_1_inputs<opset10::Negative> instead of translate_neg |
| src/frontends/pytorch/src/op/pixel_shuffle.cpp | Refactored pixel shuffle operations to use built-in OpenVINO operations and introduced helper functions for common shape manipulations |
| src/frontends/pytorch/src/op/neg.cpp | Removed entire file as the custom implementation is no longer needed |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Maxim Vafin <maxim.vafin@intel.com>
Signed-off-by: Maxim Vafin <maxim.vafin@intel.com>
bumbosiepsak
approved these changes
Jan 23, 2026
Contributor
bumbosiepsak
left a comment
bumbosiepsak
left a comment
There was a problem hiding this comment.
Quite smart! (I like usage of native operations).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Details:
Tickets: